Skip to content

ci(renovate): adding more tools for renovate - #353

Merged
cert-manager-prow[bot] merged 2 commits into
cert-manager:mainfrom
hjoshi123:fix/more-tools-deps
Sep 1, 2025
Merged

ci(renovate): adding more tools for renovate#353
cert-manager-prow[bot] merged 2 commits into
cert-manager:mainfrom
hjoshi123:fix/more-tools-deps

Conversation

@hjoshi123

Copy link
Copy Markdown
Contributor

No description provided.

@cert-manager-prow cert-manager-prow Bot added dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Aug 27, 2025
Comment thread modules/tools/00_mod.mk
Comment thread modules/tools/00_mod.mk Outdated
@hjoshi123
hjoshi123 force-pushed the fix/more-tools-deps branch from 3088a26 to 0e888eb Compare August 27, 2025 16:28
@cert-manager-prow cert-manager-prow Bot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Aug 27, 2025
@hjoshi123
hjoshi123 force-pushed the fix/more-tools-deps branch 3 times, most recently from 790bd66 to d04bece Compare August 27, 2025 17:14
Comment thread modules/tools/00_mod.mk Outdated
Comment thread modules/tools/00_mod.mk Outdated
@hjoshi123
hjoshi123 force-pushed the fix/more-tools-deps branch from d04bece to ef6ecb2 Compare August 27, 2025 23:37
Comment thread modules/tools/00_mod.mk Outdated
@cert-manager-prow cert-manager-prow Bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 28, 2025
@hjoshi123
hjoshi123 force-pushed the fix/more-tools-deps branch from ef6ecb2 to 4e73f8f Compare August 28, 2025 14:55
@cert-manager-prow cert-manager-prow Bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Aug 28, 2025
@hjoshi123
hjoshi123 force-pushed the fix/more-tools-deps branch 2 times, most recently from b1069ed to 3a80064 Compare August 28, 2025 15:48
@cert-manager-prow cert-manager-prow Bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Aug 28, 2025
Comment thread modules/tools/00_mod.mk
Comment thread modules/tools/00_mod.mk Outdated
Comment thread modules/tools/00_mod.mk Outdated
Comment thread modules/tools/00_mod.mk Outdated
Comment thread modules/tools/00_mod.mk Outdated
Signed-off-by: hjoshi123 <mail@hjoshi.me>

Update modules/tools/00_mod.mk

Co-authored-by: Erik Godding Boye <egboye@gmail.com>
Signed-off-by: Hemant Joshi <mail2hemantjoshi@pm.me>

Update modules/tools/00_mod.mk

Co-authored-by: Erik Godding Boye <egboye@gmail.com>
Signed-off-by: Hemant Joshi <mail2hemantjoshi@pm.me>

Update modules/tools/00_mod.mk

Co-authored-by: Erik Godding Boye <egboye@gmail.com>
Signed-off-by: Hemant Joshi <mail2hemantjoshi@pm.me>

Update modules/tools/00_mod.mk

Co-authored-by: Erik Godding Boye <egboye@gmail.com>
Signed-off-by: Hemant Joshi <mail2hemantjoshi@pm.me>

Update modules/tools/00_mod.mk

Co-authored-by: Erik Godding Boye <egboye@gmail.com>
Signed-off-by: Hemant Joshi <mail2hemantjoshi@pm.me>
@hjoshi123
hjoshi123 force-pushed the fix/more-tools-deps branch from 5b4cfab to 3884728 Compare August 28, 2025 19:14

@erikgb erikgb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve
/hold

To allow @inteon to decide what to do for the Vault dependency.

@cert-manager-prow cert-manager-prow Bot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. lgtm Indicates that a PR is ready to be merged. labels Aug 28, 2025
@cert-manager-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: erikgb

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@cert-manager-prow cert-manager-prow Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 28, 2025
Signed-off-by: Tim Ramlot <42113979+inteon@users.noreply.github.com>
@cert-manager-prow cert-manager-prow Bot removed the lgtm Indicates that a PR is ready to be merged. label Aug 31, 2025

@erikgb erikgb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Should we unhold this, @inteon?

@cert-manager-prow cert-manager-prow Bot added the lgtm Indicates that a PR is ready to be merged. label Aug 31, 2025
@inteon

inteon commented Sep 1, 2025

Copy link
Copy Markdown
Member

/unhold

@cert-manager-prow cert-manager-prow Bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Sep 1, 2025
@cert-manager-prow
cert-manager-prow Bot merged commit e6e83a4 into cert-manager:main Sep 1, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants